Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

Lang2SQL 프로젝트의 시각화 모듈 구조를 개선하여 viz/ 디렉토리를 utils/visualization/으로 이동
프로젝트 전체의 모듈 구조 일관성을 확보하고 시각화 관련 코드의 품질을 향상시키기 위한 리팩토링 작업
기존 기능은 모두 유지하면서 타입 힌트, docstring, 예외 처리 등을 개선하여 코드 안정성과 가독성을 향상

💬 To Reviewers (선택)

interface/core/result_renderer.py의 import 경로 변경이 올바르게 적용되었는지 검토해주세요
시각화 기능이 정상적으로 작동하는지, 특히 Streamlit에서 차트 생성이 문제없이 동작하는지 중점적으로 확인해주세요

PR Checklist

[x] 모든 import 경로가 올바르게 수정되었는지 확인
[x] 시각화 기능이 정상적으로 작동하는지 테스트

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 시각화는 잘 동작합니다!! 그런데 datahub에서 lineage 때문에 설정해놓은 값이 꼬이는지 delta.delta 이렇게 값이 들어가네요ㅜㅜ 이건 수정해보겠습니다.

image

💬 요 며칠 기여해주신 부분들을 포함해서 제가 문서화 업데이트를 한번 하겠습니다ㅋㅋ!! 감사합니다!

@ParkGyeongTae ParkGyeongTae merged commit 34ce19f into master Oct 8, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the 192-move-viz-to-utils-visualization branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants